xen/arm64: Use the correct TLBs flush instruction to nuke stage-2 TLBs
authorJulien Grall <julien.grall@arm.com>
Thu, 7 Jul 2016 18:49:08 +0000 (19:49 +0100)
committerStefano Stabellini <sstabellini@kernel.org>
Thu, 7 Jul 2016 18:57:20 +0000 (19:57 +0100)
commit730bdfa418fc8c809695ff5d96bc6f7a3b8827ba
treebf51057926af3f3b29f208140d2362f5019902ec
parent16b69afbba6b8692c96ec5a6864c35fa2fac1d36
xen/arm64: Use the correct TLBs flush instruction to nuke stage-2 TLBs

The function flush_tlb is called to invalidate the TLBs for the current
domain when the stage-2 page tables are modified.

On ARMv8, the instruction "tlbi vmalle1is" (resp. "tlbi vmalle1") will
invalidate stage 1 entries associated to the current VMID (see D4-1811 in
ARM DDI 0487A.j).

Given that an implementation is allowed to cache separately stage 1 and
stage 2 translation (see D4.7.1), the instructions will not remove stage
2 entries when the translation is not combined in a single entry.
This will result the TLBs to hold invalid entries and possibly multiple
entries using the same VA.

Use "tlbi vmalls12e1is" (resp. "tlbi vmalls12e1"), to flush both stage
1 and 2 entries when the domain p2m is changed.

Also modify flush_tlb_local to invalidate stage 1 and 2 for the local
TLBs. Note that this function is used in the instruction abort path
before translating a GVA to a IPA. As far as I understand is to avoid a
guest poisoning the DTLB when memacces is in use. We might be able to
only invalidate stage 1 entries. However, I choose the safest way for now
(i.e invalidating stage 1 and 2 entries). We would need to introduce a
new set of helpers when we will want to restrict it.

Signed-off-by: Julien Grall <julien.grall@arm.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
xen/include/asm-arm/arm64/flushtlb.h